Skip to content

Conversation

@Frando
Copy link
Member

@Frando Frando commented Nov 23, 2025

No description provided.

@n0bot n0bot bot added this to iroh Nov 23, 2025
@github-project-automation github-project-automation bot moved this to 🏗 In progress in iroh Nov 23, 2025
@Frando
Copy link
Member Author

Frando commented Nov 25, 2025

Closing in favor of #181

@Frando Frando closed this Nov 25, 2025
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in iroh Nov 25, 2025
Frando added a commit that referenced this pull request Dec 2, 2025
## Description

Based on #179 

This moves the QuicFrame generation into the `qlog` module. The public
interface for recording frames in sent packets now either takes a
`crate::Frame`, or for frame types where constructing such a frame would
have a cost, there's a specialized method on the `QlogSentPacket`.

This keeps the mapping between frames and `qlog::QuicFrame` localized
instead of spread out through the crate, and will make it much easier to
add support for more frames.

## Breaking Changes

<!-- Optional, if there are any breaking changes document them,
including how to migrate older code. -->

## Notes & open questions

<!-- Any notes, remarks or open questions you have to make about the PR.
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants